Skip to content

chore: migrate from Yarn to pnpm#29

Merged
jimmyn merged 3 commits into
masterfrom
chore/migrate-yarn-to-pnpm
Mar 30, 2026
Merged

chore: migrate from Yarn to pnpm#29
jimmyn merged 3 commits into
masterfrom
chore/migrate-yarn-to-pnpm

Conversation

@jimmyn
Copy link
Copy Markdown
Collaborator

@jimmyn jimmyn commented Mar 30, 2026 •
edited by coderabbitai Bot
Loading

Summary

  • Migrate package manager from Yarn Berry (v4) to pnpm (v10)
  • Update all devDependencies to latest compatible versions
  • Regenerate SDK from OpenAPI spec v1.8.7 (adds POSAuthToken API)

Implementation

  • Replace packageManager field: yarn@4.12.0 → pnpm@10.33.0
  • Add pnpm.onlyBuiltDependencies and pnpm.ignoredBuiltDependencies
  • Update all scripts and hooks from yarn → pnpm
  • Delete .yarnrc.yml and .yarn/ directory
  • Update .gitignore for pnpm
  • Upgrade CI actions: actions/checkout@v6, actions/cache@v5
  • Update docs (CLAUDE.md, README.md)

Test plan

  • pnpm install --frozen-lockfile succeeds
  • composer test passes (398/398)
  • CI passes on PR

Summary by CodeRabbit

Release Notes

  • Chores

    • Migrated project build system from Yarn to pnpm package manager for improved dependency management
    • Updated GitHub Actions workflow to newer major versions for enhanced CI/CD stability
    • Bumped OpenAPI Generator CLI dependency to version 2.31.0

  • Tests

    • Added new placeholder test files to expand test coverage for API endpoints and data models

jimmyn added 2 commits March 30, 2026 12:08
@jimmyn
chore: migrate from Yarn to pnpm
97b716b 
- Replace packageManager yarn@4.12.0 with pnpm@10.33.0
- Update scripts, husky hooks, release-it hooks from yarn to pnpm
- Add pnpm.onlyBuiltDependencies for openapi-generator-cli
- Update CI actions to v5/v6
- Update .gitignore for pnpm
- Delete .yarnrc.yml
- Update docs (CLAUDE.md, README.md)
@jimmyn
chore: update OpenAPI document version to 1.8.7
1e4d215 
- Regenerate SDK from latest OpenAPI spec
- Add POSAuthToken API endpoint
- Add CreatePosAuthTokenRequest and InlineObject2 models
@chatgpt-codex-connector
Copy link
Copy Markdown

chatgpt-codex-connector Bot commented Mar 30, 2026

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@jimmyn jimmyn merged commit 9180533 into master Mar 30, 2026
11 checks passed
@jimmyn jimmyn deleted the chore/migrate-yarn-to-pnpm branch March 30, 2026 10:12
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 30, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 88721a3e-0a20-4657-a7c5-2b5fdb142f19

📥 Commits

Reviewing files that changed from the base of the PR and between 80592a1 and 0cf6cd1.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (12)
  • .github/workflows/tests.yml
  • .gitignore
  • .husky/commit-msg
  • .husky/pre-commit
  • .release-it.json
  • .yarnrc.yml
  • CLAUDE.md
  • README.md
  • package.json
  • test/Api/POSAuthTokenApiTest.php
  • test/Model/CreatePosAuthTokenRequestTest.php
  • test/Model/InlineObject2Test.php
📝 Walkthrough

Walkthrough

This PR migrates the project from Yarn to pnpm as its package manager. Changes include updating CI/CD workflows to newer action versions, replacing all Yarn command invocations with pnpm equivalents across configuration and documentation files, updating package.json with pnpm configuration and a bumped openapi-generator-cli dependency, and adding placeholder test files for new API and model classes.

Changes

Cohort / File(s) Summary
Package Manager Configuration
package.json, .yarnrc.yml, .gitignore		 Migrated from Yarn to pnpm: updated packageManager field to pnpm@10.33.0, added pnpm-specific config for build dependencies, removed Yarn linker setting, replaced Yarn-specific ignore patterns with pnpm and legacy Yarn lockfile exclusions.
Build & Release Scripts
.release-it.json, .husky/commit-msg, .husky/pre-commit		 Updated all package manager invocations from yarn to pnpm in release and Git hook configurations while preserving script logic and arguments.
Documentation
README.md, CLAUDE.md		 Updated SDK build and installation instructions to use pnpm instead of yarn for all dependency and build commands.
CI/CD Workflow
.github/workflows/tests.yml		 Upgraded GitHub Actions versions: actions/checkout@v4 → actions/checkout@v6 and actions/cache@v4 → actions/cache@v5.
Build Scripts in package.json
package.json		 Updated build and build:local npm scripts to use pnpm run instead of yarn run, and bumped @openapitools/openapi-generator-cli from 2.25.2 to 2.31.0.
Test Suite
test/Api/POSAuthTokenApiTest.php, test/Model/CreatePosAuthTokenRequestTest.php, test/Model/InlineObject2Test.php		 Added new PHPUnit test classes with placeholder test methods marked as incomplete, covering POS Auth Token API and model classes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch chore/migrate-yarn-to-pnpm

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jimmyn